Skip to content

Fix async command sequence failure propagation - #4436

Closed
tomhargrave wants to merge 1 commit into
GraphiteEditor:masterfrom
tomhargrave:codex/propagate-wasm-build-failures
Closed

Fix async command sequence failure propagation#4436
tomhargrave wants to merge 1 commit into
GraphiteEditor:masterfrom
tomhargrave:codex/propagate-wasm-build-failures

Conversation

@tomhargrave

Copy link
Copy Markdown

Summary

  • return the worker result from Sequence::wait()
  • propagate child spawn failures, nonzero exits, and worker panics to callers
  • make WASM package generation fail when a generation step fails
  • add focused sequence success and failure tests

This prevents a failed wasm-bindgen or wasm-opt step from being reported as a successful WASM build.

Testing

  • cargo fmt --package cargo-run -- --check
  • cargo test -p cargo-run

@Keavon

Keavon commented Aug 15, 2026

Copy link
Copy Markdown
Member

This appears to be AI authored, in violation of our policy.

@Keavon Keavon closed this Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants